-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[OGUI-1581] Split action for taking/releasing all locks in API router #2672
[OGUI-1581] Split action for taking/releasing all locks in API router #2672
Conversation
Added Endpoint to manage all detectors, enforce Global role.
should allow access for Global user on locks ALL. Expect Taken states
…ing/releasingalllocksinAPIrouter
This middleware will add the detectorid object to the requests
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @pepijndik !
Very nice and quick PR, especially well done on the testing side!
I added a few minor comments for you to have a look and fix
…ing/releasingalllocksinAPIrouter
Control/test/lib/middleware/mocha-addDetectorId.middleware.test.js
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please have a look at one comment I added but also at the GitHub Code Scanning comments
I have JIRA issue created
Added middleware to append detectorid, and endpoint for action of all detectors